Seconde correction et explication des nouvelles alertes rencontrées :

• "Final newline missing". Il doit y avoir une ligne vide à la fin du programme.
• "Line too long". Un retour à la ligne est nécessaire.
• "Unused import [...]". Afin d'accroître les performances et la sécurité du programme, il vaut mieux supprimer ces imports qui ne sont pas utilisés.

Voici une liste de tous les fichiers corrigés jusqu'à présent :

├─ /Discord-Bot-main
→    ├─ /Bot
→→        ├─ /ur_bot.py

├─ /src
→    ├─ /bot
→→        ├─ /__init__.py
→→        ├─ /cog_about.py
→→        ├─ /cog_general.py
→→        ├─ /settings.py
→→        ├─ /strings.py
→→        ├─ /urbot.py
→    ├─ /urpy
→→        ├─ /localization.py
→    ├─ /start.py

├─ /bot
→    ├─ /extends
→→        ├─ /base
→→→            ├─ /base.py

Explication des nouvelles alertes rencontrées :

• "Shadows built-in name [...]" : le nom donné à l'objet (fonction, variable, etc.) est le même qu'un autre objet déjà intégré à l'installation de python. Par exemple, il est interdit de donner à une variable le nom 'credits', car il s'agit d'une fonction dite "built-in", dont aucun import n'est nécessaire pour y accéder. C'est pourquoi il faut renommer cette variable.è
• "Shadows name [...] from outer scope" : le principe est le même qu'au-dessus, à la différence que le nom donné à l'objet interfère avec un autre objet dont nous avons nous même défini ou importé. C'est pourquoi il faut renommer le premier objet.
• "Method [...] may be 'static' " : c'est une erreur indiquant que la méthode [...] pourrait être définie comme une méthode statique. Cela signifie que la méthode ne fait pas référence aux attributs de l'instance de la classe (via le paramètre 'self') et peut donc être rendue plus flexible en étant définie comme '@staticmethod'.
• "Module name [...] doesn't conform to snake_case naming style". Le nom du fichier doit être changé pour respecter la convention 'snake_case'. Par exemple : 'UR-BOT' devient 'ur_bot'.

Post-scriptum : Au dernier commit, il est écrit qu'un fichier a été crée et qu'un autre a été supprimé. En fait, les deux sont exactement les mêmes. Il n'y a donc que le nom du fichier qui a changé. Par extension, ses exportations ont repris le nouveau nom, qui respecte les normes codacy.

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

^{🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.}

À noter :

• Les erreurs rencontrées par GitGuardian concernent la présence d'URL de Webhooks Discord, inclues directement dans le code de 'Discord-Bot-main/Bot/event.py'.
• Ce ticket permet de corriger le code du projet. Sans ce travail, certains imports ne seraient pas fonctionnels ; des variables ne seraient jamais utilisés ; et d'autres choses réduiraient la qualité du code, voire l'arrêteraient sur un message d'erreur. Par exemple, en appelant une fonction provenant d'un autre programme, il faut spécifier le bon chemin (ou le chemin en entier) vers cette dernière pour l'importer de manière correcte. Dans le cas contraire, l'appel de la fonction ne fonctionnerait pas et le bot ne répond pas aux exigences désirées.

Voici une liste de tous les fichiers corrigés jusqu'à présent ; en gras, sont les nouveaux fichiers corrigés et en italique, ceux ajoutés à l'arborescence car ils ne possèdent pas d'erreur :

├─ /.github
→ ├─ /workflows
→→ ├─ /Add-issue-to-project.yml
→→ ├─ /codacy.yml
→→ ├─ /codeql.yml
→→ ├─ /dependency-review.yml

├─ /archives
→ ├─ /functions.py

├─ /bot/

├─ /Discord-Bot-main
→ ├─ /Bot
→→ ├─ /debug_bot.py
→→ ├─ /event.py
→→ ├─ /requirements.py
→→ ├─ /ur_bot.py
→ ├─ /Install/
→ ├─ /README.md

├─ /src
→ ├─ /bot
→→ ├─ /init.py
→→ ├─ /cog_about.py
→→ ├─ /cog_general.py
→→ ├─ /settings.py
→→ ├─ /strings.py
→→ ├─ /urbot.py
→ ├─ /urpy
→→ ├─ /localization.py
→ ├─ /start.py

├─ /bot
→ ├─ /extends
→→ ├─ /base
→→→ ├─ /base.py

Explication des nouvelles alertes rencontrées :

• "Wrong hanging indentation (remove [...] space[s])". L'erreur se produit lorsque l'indentation d'une ligne de continuation n'est pas alignée correctement avec la ligne précédente.
• "Remove redundant parentheses". Cela signale des parenthèses inutiles à un endroit du code, réduisant logiquement les performances du code.
• "Parameter '[paramètre]' value is not used". Le paramètre n'est jamais utilisé, et retire des performances au programme.
• "Usages of [...] Nothing found in 'Project Files'". Le fichier, la fonction ou la variable (etc.) n'est jamais utilisé dans le projet. Dans tous les cas, avant de manipuler quoi que ce soit, il faut toujours vérifier si le code n'est pas utilisé ailleurs. Par exemple : 'Discord-Bot-main/Bot/requirements.txt' est utilisé par docker-compose. Pourtant, cette manipulation n'est pas affichée par l'IDE quand on cherche les utilisations du fichier.
• "Cannot resolve file '[...]'". Ce problème a été identifié dans le fichier 'Discord-Bot-main/README.md', dont la ligne > **Télécharger** ![Docker](docker.com) est invalide, car il ne faut pas mettre de point d'exclamation. En effet, ce dernier indique au code d'aller chercher une image au chemin spécifié ; soit ici, sur le web. Pourtant, le développeur cherchait à indiquer un hyperlien. Pour ce faire, il suffit d'écrire la même ligne, sans point d'exclamation. Ainsi, elle devient : > **Télécharger** [Docker](docker.com)

Tout le programme du répertoire Bot_Base a été corrigé. Il ne s'agissait pas de créer de nouveaux codes. En revanche, modifier celui existant pour que ces derniers fonctionnent comme désirés. Donc, soyez assurez que la manière dont les programmes agissent ne changent pas.

Voici une liste de tous les fichiers corrigés jusqu'à présent ; en gras, sont les nouveaux fichiers corrigés et en italique, ceux ajoutés à l'arborescence car ils ne possèdent pas d'erreur :

├─ /.github
→ ├─ /workflows/

├─ /archives/

├─ /bot/

├─ /Discord-Bot-main/

├─ /img

├─ /src
→ ├─ /bot/
→ ├─ /locale/
→ ├─ /urpy
→→ ├─ /templates/
→→ ├─ /init.py
→→ ├─ /get_ressources.py
→→ ├─ /help.py
→→ ├─ /localization.py
→→ ├─ /requirements.txt
→→ ├─ /utils.py
→→ ├─ */xml.py
→ ├─ /Bot_Base.service
→ ├─ /start.py

├─ /.gitignore
├─ /add_repo.sh
├─ /changelog.txt
├─ /credits.md
├─ /install.sh
├─ /README.md
├─ /SECURITY.md
├─ /start.sh
├─ /updateBot.sh

Explication des nouvelles alertes rencontrées :

• "Invalid escape sequence" : Il suffit donc de rajouter un deuxième slash inversé ('').
• "Single character alternation in RegExp". C'est pourquoi nous retirons l'espace en trop. Le code précité devient *(?:\n|)
• "Using an f-string that does not have any interpolated variables (f-string-without-interpolation)". Un 'f-string' correspond à une chaîne de caractères précédées par un préfixe 'f' (pour 'format'). C'est ce dernier qui permet à la chaîne d'accueillir des variables prédéfinies, par exemple :

a = 3
b = 9
print(f"{a} * {b} = {a*b}")
>>> 27

L'erreur ci-dessus indique l'utilisation d'une telle chaîne de caractères, qui n'appelle pourtant pas de variable. C'est pourquoi il vaut mieux supprimer le préfixe et ainsi optimiser le code.